-
Notifications
You must be signed in to change notification settings - Fork 25.6k
drop_null_columns requires json or yaml format #116432
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Adding an additional description which notes that using the drop_null_columns parameter requires the format to be set to yaml or json. If these formats are not used, null columns will still be shown.
Documentation preview: |
@asmith-elastic please enable the option "Allow edits and access to secrets by maintainers" on your PR. For more information, see the documentation. |
Pinging @elastic/es-docs (Team:Docs) |
Pinging @elastic/es-analytical-engine (Team:Analytics) |
Sounds a bit like a bug to be honest. OTOH, for things like Arrow we can't know if the column is all the way null without getting all the output which is kind of a thing we'd like to avoid. |
What about cbor and smile? |
@nik9000 - Good catch! It does look like Given that this sounds like a bug, would you recommend I open up an associated issue to track if the other human readable options, such as csv, tsv, & txt has the same sort of limitations as |
Updating the wording to include CBOR, JSON, SMIL, and YAML formats.
@nik9000 are you happy to merge this docs PR? |
Important Elastic documentation is migrating to Markdown for version 9.0+. See the migration guide for details. ℹ️ What's happening?
What do I need to do?For <=8.x docs:
For 9.0+ docs:Option 1:
Option 2:
💡 Need help?
|
Ah! We've fixed |
This request will add an additional description to the
drop_null_columns
query parameter which notes that the parameter requires the format to be set toyaml
orjson
. If any other format is being used, null columns will still be shown.